Skip to content

fix qodana python version#21

Closed
xprilion wants to merge 2 commits into
mainfrom
dev
Closed

fix qodana python version#21
xprilion wants to merge 2 commits into
mainfrom
dev

Conversation

@xprilion
Copy link
Copy Markdown
Owner

This pull request introduces several improvements and minor fixes across the codebase, focusing on dependency management, resource cleanup, and import handling. The most notable changes are grouped below.

Dependency and Environment Configuration:

  • Added explicit python-version: "3.12" to the Qodana linter configuration in qodana.yaml to ensure consistent linting and compatibility across environments.

Resource Management and Cleanup:

  • Improved resource cleanup in backend/openmlr/tools/compute_tools.py by ensuring that the SandboxManager (sm) is always defined and properly destroyed in error cases during the _handle_probe function. [1] [2]

Code Quality and Import Handling:

  • Updated import logic in backend/openmlr/services/session_manager.py to use absolute imports for database operations, improving code clarity and maintainability.

Testing Improvements:

  • Cleaned up unreachable code in the test_cancelled_returns_none test by removing a dead code path in the mock stream generator.

@github-actions
Copy link
Copy Markdown

Qodana for Python

173 new problems were found

Inspection name Severity Problems
Unbound local variables 🔶 Warning 7
Unused imports 🔶 Warning 2
Redundant character escape 🔶 Warning 1
Unused local symbols ◽️ Notice 67
Accessing a protected member of a class or a module ◽️ Notice 43
Unclear exception clauses ◽️ Notice 30
Shadowing names from outer scopes ◽️ Notice 10
Method is not declared static ◽️ Notice 6
PEP 8 naming convention violation ◽️ Notice 4
Non-optimal list declaration ◽️ Notice 2
An instance attribute is defined outside init`` ◽️ Notice 1

☁️ View the detailed Qodana report

Contact Qodana team

Contact us at qodana-support@jetbrains.com

@xprilion xprilion closed this Apr 27, 2026
@xprilion xprilion deleted the dev branch April 27, 2026 09:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant